Skip to content

README: update 2 broken hyperlinks - #5007

Merged
urvangjoshi merged 2 commits into
AOMediaCodec:mainfrom
urvangjoshi:readme_links
May 26, 2026
Merged

README: update 2 broken hyperlinks#5007
urvangjoshi merged 2 commits into
AOMediaCodec:mainfrom
urvangjoshi:readme_links

Conversation

@urvangjoshi

@urvangjoshi urvangjoshi commented May 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Also remove the [TOC], that is specific to Gitlab.

@urvangjoshi
urvangjoshi enabled auto-merge (squash) May 23, 2026 00:47
@FooIbar

FooIbar commented May 24, 2026

Copy link
Copy Markdown

Should the [TOC] in README be removed since it's GitLab-specific?

@urvangjoshi

urvangjoshi commented May 26, 2026

Copy link
Copy Markdown
Contributor Author

Should the [TOC] in README be removed since it's GitLab-specific?

Thank you for your suggestion! Removed it in the updated PR.

andrey-norkin
andrey-norkin approved these changes May 26, 2026
View reviewed changes

@andrey-norkin andrey-norkin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but perhaps, we could also add a link to the public CTC document in addition to the CTC sequences.

@urvangjoshi
urvangjoshi merged commit c1a34e8 into AOMediaCodec:main May 26, 2026
109 checks passed
@urvangjoshi

urvangjoshi commented May 26, 2026

Copy link
Copy Markdown
Contributor Author

LGTM, but perhaps, we could also add a link to the public CTC document in addition to the CTC sequences.

Thanks for the review @andrey-norkin . Looks like PR was auto-merged after approval.
I'll prepare a separate PR to add link to CTC doc, if that's desired.

@urvangjoshi
urvangjoshi deleted the readme_links branch May 26, 2026 20:30
@urvangjoshi

urvangjoshi commented May 26, 2026

Copy link
Copy Markdown
Contributor Author

LGTM, but perhaps, we could also add a link to the public CTC document in addition to the CTC sequences.

Thanks for the review @andrey-norkin . Looks like PR was auto-merged after approval. I'll prepare a separate PR to add link to CTC doc, if that's desired.

@andrey-norkin : here you go: #5012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andrey-norkin andrey-norkin andrey-norkin approved these changes
@agrange agrange Awaiting requested review from agrange

Assignees

No one assigned

Labels

cla-signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@urvangjoshi @FooIbar @andrey-norkin @aom-bot